Skip to content

fix(node-on-headers): CVE-2025-7339#1

Open
deepin-ci-robot wants to merge 1 commit intomasterfrom
fix/CVE-2025-7339
Open

fix(node-on-headers): CVE-2025-7339#1
deepin-ci-robot wants to merge 1 commit intomasterfrom
fix/CVE-2025-7339

Conversation

@deepin-ci-robot
Copy link
Copy Markdown
Contributor

CVE 修复

CVE ID: CVE-2025-7339

问题描述:
on-headers versions <1.1.0 存在一个 bug,当数组传递给 response.writeHead() 时,可能导致响应头被意外修改。

修复内容:

  • 修复 setHeadersFromArray 函数以正确处理 1D 和 2D header 数组
  • 添加对 malformed 数组的检测

上游修复: jshttp/on-headers@c6e3849

修复版本: 1.1.0

当前版本: 1.0.2

策略: patch (backport)


Generated-By: glm-5.1
Co-Authored-By: hudeng hudeng@deepin.org

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

TAG Bot

TAG: 1.0.2-2deepin1
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot deepin-ci-robot requested a review from justforlxz May 6, 2026 06:21
@deepin-ci-robot
Copy link
Copy Markdown
Contributor Author

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zeno-sole for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-ci-robot deepin-ci-robot force-pushed the fix/CVE-2025-7339 branch 2 times, most recently from 520b274 to f76f26e Compare May 6, 2026 09:31
Fix array handling in setHeadersFromArray to prevent header
modification issues when arrays are passed to response.writeHead().

This fix properly handles both 1D and 2D header arrays, and includes
upstream test coverage for 1D flat and malformed array handling.

Upstream: jshttp/on-headers@c6e3849

Generated-By: deepseek-v4-pro
Co-Authored-By: hudeng <hudeng@deepin.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants